Skip to content

build: show CI results on PR#64483

Open
avivkeller wants to merge 1 commit into
nodejs:mainfrom
avivkeller:ci-results-on-pr
Open

build: show CI results on PR#64483
avivkeller wants to merge 1 commit into
nodejs:mainfrom
avivkeller:ci-results-on-pr

Conversation

@avivkeller

@avivkeller avivkeller commented Jul 14, 2026

Copy link
Copy Markdown
Member

Fixes #32306
Fixes #64333

cc @nodejs/jenkins-admins this requires updating the node-test-pull-request to call the like https://api.github.com/repos/{owner}/{repo}/dispatches with pr and id (the pull request to comment on, and the Jenkins run ID, respectively)

@avivkeller avivkeller added the blocked PRs that are blocked by other issues or PRs. label Jul 14, 2026
@nodejs-github-bot

nodejs-github-bot commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/actions

@nodejs-github-bot nodejs-github-bot added meta Issues and PRs related to the general management of the project. tools Issues and PRs related to the tools directory. labels Jul 14, 2026
@avivkeller
build: show CI results on PR
03f52bd 
Signed-off-by: Aviv Keller <me@aviv.sh>
@avivkeller

avivkeller commented Jul 14, 2026

Copy link
Copy Markdown
Member Author

As an example, the CI failures of the most recent Jenkins failed run are:

Failures in job https://ci.nodejs.org/job/node-test-pull-request/74799/

debian12-x64

See failures on test-ibm-debian12-x64-1: 
error: could not apply 61890720c8a... add readme and initial code
hint: Resolve all conflicts manually, mark them as resolved with
hint: "git add/rm <conflicted_files>", then run "git rebase --continue".
hint: You can instead skip this commit: run "git rebase --skip".
hint: To abort and get back to the state before "git rebase", run "git rebase --abort".
Could not apply 61890720c8a... add readme and initial code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@efekrskl efekrskl efekrskl approved these changes
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

blocked PRs that are blocked by other issues or PRs. meta Issues and PRs related to the general management of the project. tools Issues and PRs related to the tools directory.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Show CI failure details to non-collaborator PR authors RFC: update Pull Requests with summary of CI failures

3 participants

@avivkeller @nodejs-github-bot @efekrskl